Skip to content

feat(mexp): Onboarding Modelexperiments#1547

Open
paul-sffrth wants to merge 40 commits into
stackitcloud:mainfrom
paul-sffrth:feat/mlflow
Open

feat(mexp): Onboarding Modelexperiments#1547
paul-sffrth wants to merge 40 commits into
stackitcloud:mainfrom
paul-sffrth:feat/mlflow

Conversation

@paul-sffrth

Copy link
Copy Markdown
Member

Description

This is the onboarding PR for integration STACKIT Modelexperiments into the STACKIT Terraform Provider.

It adds for customer the ability to:

  • Create, Read, Update and Delete Modelexperiments Instances
  • Create, Read, Update and Delete Mdelexperiments Instance Tokens

Checklist

  • Issue was linked above
  • Code format was applied: make fmt
  • Examples were added / adjusted (see examples/ directory)
  • Docs are up-to-date: make generate-docs (will be checked by CI)
  • Unit tests got implemented or updated
  • Acceptance tests got implemented or updated (see e.g. here)
  • Unit tests are passing: make test (will be checked by CI)
  • No linter issues: make lint (will be checked by CI)

@paul-sffrth
paul-sffrth requested a review from a team as a code owner June 29, 2026 09:24
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

This PR was marked as stale after 7 days of inactivity and will be closed after another 7 days of further inactivity. If this PR should be kept open, just add a comment, remove the stale label or push new commits to it.

@github-actions github-actions Bot added the Stale PR is marked as stale due to inactivity. label Jul 8, 2026
@Fyusel Fyusel removed the Stale PR is marked as stale due to inactivity. label Jul 8, 2026
Comment thread examples/resources/stackit_modelexperiments_token/resource.tf
Comment thread stackit/internal/services/modelexperiments/instance/description.md Outdated
Comment thread stackit/internal/services/modelexperiments/instance/resource.go Outdated
Comment thread stackit/internal/services/modelexperiments/instance/resource.go
Comment thread stackit/internal/services/modelexperiments/token/description.md Outdated
Comment thread stackit/internal/services/modelexperiments/utils/util.go Outdated
Comment thread stackit/internal/services/modelexperiments/token/resource.go Outdated
Comment thread stackit/internal/services/modelexperiments/instance/resource.go Outdated
Comment thread stackit/internal/services/modelexperiments/token/resource.go Outdated
@@ -0,0 +1,610 @@
package token

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A datasource implements the GET request. There is always a datasource present to read a resource. By adding this datasource at least the metadata can be read without using another tool like the CLI or Portal to obtain those informations.

Question for my understanding:
There is an ID and a name returned in the GET request. Can this not be used to reference the token?
API Docs

Comment thread stackit/internal/services/modelexperiments/token/resource.go Outdated
Comment thread stackit/internal/services/modelexperiments/token/resource.go Outdated
Comment thread stackit/internal/services/modelexperiments/token/resource.go
Comment thread stackit/internal/services/modelexperiments/instance/resource.go Outdated
Comment thread stackit/internal/services/modelexperiments/instance/resource.go Outdated
Comment thread stackit/internal/services/modelexperiments/token/resource.go Outdated
Comment thread stackit/internal/services/modelexperiments/modelexperiments_acc_test.go Outdated
@Fyusel

Fyusel commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

After those comments are resolved I will start to run the acceptance tests and manual testing

},
"region": schema.StringAttribute{
Description: descriptions["region"],
Optional: true,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs to be added:
// the region cannot be found, so it has to be passed
Computed: true

},
"region": schema.StringAttribute{
Description: descriptions["region"],
Optional: true,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs to be added:
// the region cannot be found, so it has to be passed
Computed: true

deleted_experiment_retention = "30d"
labels = {
label = "Example label"
}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This label example does not work (regex validation error)

description = "Example description"
deleted_experiment_retention = "30d"
labels = {
label = "Example label"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This label example does not work (regex validation error)

description = "Example token description"
ttl_duration = "1h"
labels = {
label = "Example label"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This label example does not work (regex validation error)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants